-
Notifications
You must be signed in to change notification settings - Fork 0
Remove accidentally committed Docusaurus build artifacts, temporary scripts, and fix .gitignore #6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
|
Important Review skippedBot user detected. To trigger a single review, invoke the You can disable this status message by setting the Comment |
Co-authored-by: gkorland <753206+gkorland@users.noreply.github.com>
|
@coderabbitai review |
✅ Actions performedReview triggered.
|
|
@copilot please fix the spelling issues in the CI |
Co-authored-by: gkorland <753206+gkorland@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR removes accidentally committed build artifacts from the website/docs/ directory and updates the .gitignore file to prevent future commits of Docusaurus-generated files. The changes clean up the repository by removing 173 duplicate documentation files that were build output, while preserving the source documentation in the root directory.
Changes:
- Removed entire
website/docs/directory containing Docusaurus build artifacts - Updated
.gitignorewith comprehensive exclusions for Docusaurus, dependencies, and development files - Updated
website/docusaurus.config.tsandwebsite/sidebars.tsto reference root directory as docs source
Reviewed changes
Copilot reviewed 107 out of 164 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
website/docs/* |
Removed all 173 build artifact files that duplicated root documentation |
.gitignore |
Added comprehensive exclusions for build outputs, dependencies, and editor files |
website/docusaurus.config.ts |
Configured to use root directory as documentation source |
website/sidebars.ts |
Removed reference to non-existent 'staging' document |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@copilot please also delete from the main folder the python script that are not needed and were just been used temporary in the development process, hint scripts that are poing to a local folder on danshalev machine |
Co-authored-by: gkorland <753206+gkorland@users.noreply.github.com>
Co-authored-by: gkorland <753206+gkorland@users.noreply.github.com>
Co-authored-by: gkorland <753206+gkorland@users.noreply.github.com>
Fixed in commit d80afdd. Added technical terms (Docusaurus, MDX, admonition, admonitions, gitignore, webpack, kramdown, JSX) to
Fixed in commit d80afdd. Deleted 11 temporary Python scripts and 2 output text files that contained hardcoded paths to danshalev's local machine:
|
The
website/docs/directory contained 173 duplicate documentation files that were accidentally committed build artifacts. Root directory files are the actual source. Additionally, removed temporary Python development scripts with hardcoded local paths.Changes
Repository Cleanup
website/docs/directory entirely (173 files)/Users/danshalev/docs-staging/final_results.txt,output.txt)Configuration Updates
website/docusaurus.config.ts: Addedpath: '../'to read docs from root, with exclude patterns for node_modules and non-doc fileswebsite/sidebars.ts: Removed reference to non-existent 'staging' document.gitignore: Added comprehensive exclusions for Docusaurus artifacts, dependencies, OS files, editor files.wordlist.txt: Added technical terms (Docusaurus, MDX, admonition, admonitions, gitignore, webpack, kramdown, JSX) to fix spelling CIMDX Syntax Fixes
Fixed compilation errors in 11 files:
<note>→:::note)<br>→<br/>,<img>→<img/>){tokenId}→\{tokenId\})<details>blocks with newlinesResult
Build artifacts now properly excluded from version control. Docusaurus compiles successfully with webpack passing and static site generated in
website/build/. All temporary development files removed. Spelling CI issues resolved.Original prompt
This pull request was created from Copilot chat.
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.